Skip to content

fix: debug log for pending messages count in RedisStreamBroker #96

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

FlorianLB
Copy link
Contributor

The debug message is wrong and always display Found 3 pending messages in stream XXX.

It's because the result of xautoclaim is a tuple with 3 values. Messages are in the second value.

@s3rius
Copy link
Member

s3rius commented Jun 28, 2025

Oh god. Thanks for the request.

@FlorianLB
Copy link
Contributor Author

Anything missing for merging ?

@s3rius
Copy link
Member

s3rius commented Jul 25, 2025

No. Thanks for making the request.

@s3rius s3rius merged commit 7160ca1 into taskiq-python:main Jul 25, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants